[Tarantool-patches] [PATCH luajit 3/4][v2] OSX/iOS/ARM64: Fix generation of Mach-O object files.

Sergey Bronnikov sergeyb at tarantool.org
Thu Jun 13 18:50:47 MSK 2024


Sergey,

On 16.04.2024 18:29, Sergey Bronnikov via Tarantool-patches wrote:
<snipped>
>> Minor: we can use `subtest.name` as the definition of the `hw_arch` in the
>> `build_and_check_mach_o()`, so it helps to avoid duplication of arch
>> usage.
>>
>> Matter of taste.
>> Feel free to ignore.
> ignored

removed additional argument with `hw_arch` and replaced with subtest.name:


@@ -293,7 +295,8 @@ local SUM_CPUSUBTYPE = {
  -- $ luajit -b -o osx -a arm64 empty.lua empty.o
  -- $ lipo -archs empty.o
  -- x86_64 arm64
-local function build_and_check_mach_o(subtest, hw_arch)
+local function build_and_check_mach_o(subtest)
+  local hw_arch = subtest.name
    assert(hw_arch == 'arm' or hw_arch == 'arm64')

    subtest:plan(4)
@@ -341,7 +344,7 @@ local function build_and_check_mach_o(subtest, hw_arch)
               'cpusubtype is correct in Mach-O')
  end

-test:test('arm', build_and_check_mach_o, 'arm')
-test:test('arm64', build_and_check_mach_o, 'arm64')
+test:test('arm', build_and_check_mach_o)
+test:test('arm64', build_and_check_mach_o)

  test:done(true)

>>> +
>>> +test:done(true)
>>> -- 
>>> 2.34.1
>>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20240613/f306a8b7/attachment.htm>


More information about the Tarantool-patches mailing list