[all-commits] [llvm/llvm-project] 4ae23b: [NVPTX] Correct the condition to print "\t}\n" for...
Fangrui Song via All-commits
all-commits at lists.llvm.org
Fri Jun 28 14:15:59 PDT 2024
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 4ae23bcca144b542f16d45acc8f270e156e2fa4e
https://github.com/llvm/llvm-project/commit/4ae23bcca144b542f16d45acc8f270e156e2fa4e
Author: Fangrui Song <i at maskray.me>
Date: 2024-06-28 (Fri, 28 Jun 2024)
Changed paths:
M llvm/lib/MC/MCAsmStreamer.cpp
M llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXTargetStreamer.cpp
M llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXTargetStreamer.h
Log Message:
-----------
[NVPTX] Correct the condition to print "\t}\n" for DWARF sections
`NVPTXTargetStreamer::changeSection` needs `CurSection` to properly
print "\t}\n" for DWARF sections.
db48f1a1764023f8efeb055e343b967d1eb37d19 added
`MCStreamer::changeSection` before `TS->changeSection`,
making `CurSection == Section` when `getCurrentSectionOnly` switches to
`CurFrag` (626eef5ecf92e98cbfccfa6134e0a760e7592813).
cf311a1131b9aef3e66b2a20ad49cfc77212754b did fix the bug, but the
mechanism is confusing. Fix it using nullable getCurrentSectionOnly().
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list