[all-commits] [llvm/llvm-project] 7939ce: [builtins] Move cfi start's after the symbol name ...

Jon Roelofs via All-commits all-commits at lists.llvm.org
Fri Nov 17 14:33:50 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7939ce39dac0078fef7183d6198598b99c652c88
      https://github.com/llvm/llvm-project/commit/7939ce39dac0078fef7183d6198598b99c652c88
  Author: Jon Roelofs <jonathan_roelofs at apple.com>
  Date:   2023-11-17 (Fri, 17 Nov 2023)

  Changed paths:
    M compiler-rt/lib/builtins/assembly.h

  Log Message:
  -----------
  [builtins] Move cfi start's after the symbol name [NFC]

... in preparation for diagnosing improperly nested .cfi regions.

See https://reviews.llvm.org/D155245


  Commit: 4323da926f12672daec7f59384bd153a7cf28674
      https://github.com/llvm/llvm-project/commit/4323da926f12672daec7f59384bd153a7cf28674
  Author: Jon Roelofs <jonathan_roelofs at apple.com>
  Date:   2023-11-17 (Fri, 17 Nov 2023)

  Changed paths:
    M lld/test/COFF/gc-dwarf-eh.s
    M llvm/lib/MC/MCParser/AsmParser.cpp
    A llvm/test/MC/AArch64/cfi-bad-nesting.s

  Log Message:
  -----------
  [MC][AsmParser] Diagnose improperly nested .cfi frames

This showed up when simplifying some large testcase, where the cfi directives
became out of sync with the proc's they enclose.

rdar://111459507

Differential revision: https://reviews.llvm.org/D155245

This reverts commit 4172fcc1ebbe0a7b699bfcbdaae9d5f688b62b09.


Compare: https://github.com/llvm/llvm-project/compare/a30e9a1a57bd...4323da926f12


More information about the All-commits mailing list