[all-commits] [llvm/llvm-project] daf7d7: [AArch64][GlobalISel] Correct function evaluation ...

Jessica Paquette via All-commits all-commits at lists.llvm.org
Tue Feb 23 16:23:07 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: daf7d7f0dc283873f40f3500b20f02bc2cde701d
      https://github.com/llvm/llvm-project/commit/daf7d7f0dc283873f40f3500b20f02bc2cde701d
  Author: Jessica Paquette <jpaquette at apple.com>
  Date:   2021-02-23 (Tue, 23 Feb 2021)

  Changed paths:
    M llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerLowering.cpp

  Log Message:
  -----------
  [AArch64][GlobalISel] Correct function evaluation order in applyINS

The order in which the nested calls to Builder.buildWhatever are
evaluated in differs between GCC and Clang.

This caused a bot failure because the MIR in the testcase was
coming out in a different order than expected.

Rather than using nested calls, pull them out in order to fix the
order of evaluation.




More information about the All-commits mailing list