[PATCH] D112595: [IRSymTab] Mark __stack_chk_guard used

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 27 14:42:55 PDT 2021


MaskRay added a comment.

> `__stack_chk_guard` is a user-defined variable

Can be more accurate. For some architectures glibc define the variable (https://sourceware.org/bugzilla/show_bug.cgi?id=26817)

> There are a couple other symbols handled specially in Internalize.cpp (stack_chk_fail, and ssp_canary_word for AIX), should the all be handled uniformly?

Echo this request. Would'be great to make the logic centralized, but may not be in this patch is you think not fit.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112595/new/

https://reviews.llvm.org/D112595



More information about the llvm-commits mailing list